From: Lukáš Tyrychtr <ltyrycht@redhat.com>
Date: Thu, 11 May 2023 09:19:36 +0000 (+0200)
Subject: a11y: Don't create a parent loop for quick settings sliders
X-Git-Tag: archive/raspbian/50.5-1+rpi1~1^2~337^2~1^2~156
X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/%22/%22http:/www.example.com/cgi/%22?a=commitdiff_plain;h=a5cf08ac556dc9c339c03bb8d7fb24e5c6b56d38;p=gnome-shell.git

a11y: Don't create a parent loop for quick settings sliders

Rather than returning the parent container of the slider, which presents
itself as the slider, return the parent of the slider container.

https://gitlab.gnome.org/GNOME/gnome-shell/-/issues/6686

Part-of: <https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/2762>
---

diff --git a/js/ui/quickSettings.js b/js/ui/quickSettings.js
index 7f08d6f75..9016fedf6 100644
--- a/js/ui/quickSettings.js
+++ b/js/ui/quickSettings.js
@@ -323,7 +323,10 @@ var QuickSlider = GObject.registerClass({
         });
         box.add_child(sliderBin);
 
-        sliderBin.set_accessible(this.slider.get_accessible());
+        // Make the slider bin transparent for a11y
+        const sliderAccessible = this.slider.get_accessible();
+        sliderAccessible.set_parent(sliderBin.get_parent().get_accessible());
+        sliderBin.set_accessible(sliderAccessible);
         sliderBin.connect('event', (bin, event) => this.slider.event(event, false));
 
         this._menuButton = new St.Button({